Fixed Double scan issue Cudi sale tool #926
Merged
+4
−3
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
De code die wordt meegegeven aan de browser zit in "Litus/public/_assetic/sale_js.js" (dus niet in de CudiBundle) => Dit is een geminimaliseerde versie van verschillende sale tool scriptjes , momenteel is dit bestand READ-ONLY je moet dus best ook met chmod de lees en schrijf rechten van dit bestand aanpassen, anders worden je veranderingen in de andere scriptjes niet meegegeven aan de browser
sudo chmod 666 public/_assetic/sale_js.js
(standaard rechten zijn 644, voor moest je ze daarna willen terugbrengen naar hun originele staat)
Bij elke refresh van de browser wordt sale_js.js opnieuw gegenereerd vanuit de "bron scriptjes" wanneer je dus de schrijfrechten hebt aangepast kan je dus aanpassingen doen in de "bron scriptjes", je aanpassingen maken rechtstreeks in sale_js.js is dus nutteloos, want deze worden meteen overschreden wanneer sale_js.js opnieuw wordt gegenereerd
Het probleem zat em in saleInterface.js => _gotBarcode() => der is 2 keer dezelfde logica die de "add Article" knop zoekt en dan daarop klikt. waardoor er 2 keer een article wordt toegevoegd.
Op eerste zicht doen beide implementaties exact hetzelfde, maar ze doen het allebei op een ietsjes andere manier. ik gok dat dit is omdat één implementatie op zichzelf niet alle edge-cases vangt en het dus 2 keer wordt gedaan, maar ben niet een JS-expert dus kan het niet met zekerheid zeggen.
Nu is het zo dat als de eerste methode de "add Article" knop vindt en erop klikt de 2de methode niet meer wordt uitgevoerd. Dan heb je nog steeds de 2de methode als "safety" in het geval van edge cases.